-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use basespace CLI to download run data #13
Conversation
The old method of downloading data created a directory within the run directory, which is different than this current method. We need to ensure that we can still demux the data. This may involve changing the demux scripts with the updated directory structure. The contents of the directories are the same. New download app: $ tree new_method -L 1
new_method
├── <run_id>.json
├── Config
├── Data
├── InstrumentAnalyticsLogs
├── InterOp
├── Logs
├── Recipe
├── RTAComplete.txt
├── RTAConfiguration.xml
├── RTALogs
├── RTARead1Complete.txt
├── RTARead2Complete.txt
├── RTARead3Complete.txt
├── RTARead4Complete.txt
├── RunCompletionStatus.xml
├── RunInfo.xml
├── RunParameters.xml
└── SampleSheet.csv
7 directories, 11 files Old download app: $ tree old_method -L 2
old_method
├──<run_id>
│ ├── Config
│ ├── Data
│ ├── InstrumentAnalyticsLogs
│ ├── InterOp
│ ├── Logs
│ ├── Recipe
│ ├── RTAComplete.txt
│ ├── RTAConfiguration.xml
│ ├── RTALogs
│ ├── RTARead1Complete.txt
│ ├── RTARead2Complete.txt
│ ├── RTARead3Complete.txt
│ ├── RTARead4Complete.txt
│ ├── RunCompletionStatus.xml
│ ├── RunInfo.xml
│ ├── RunParameters.xml
│ └── SampleSheet.csv
└── nohup.out
8 directories, 11 files |
After some more investigation, this does not cause any issues with the demux app. Now instead of running the following: singularity run \
-B $resource_dir:/opt/resources -B $run_dir:/opt/analysis -B $bcl_dir:/opt/data \
miptools.sif --app demux -s $sample_list -p $platform where singularity run \
-B $resource_dir:/opt/resources -B $run_dir:/opt/analysis -B $run_dir:/opt/data \
miptools.sif --app demux -s $sample_list -p $platform We will get the same output. We may additionally want to consider slightly changing the demux script so that we no longer need to bind |
This PR replaces the old python script for downloading data from the Illumina BaseSpace Sequence Hub.
Resolves #8